Skip to content

Conversation

@graphitefriction
Copy link
Contributor

  • fix titles and remove extra lines
  • remove quote blocks and add admonitions
  • remove indents/spaces from tables and add implicit headers
  • remove unconstrained inline formatting
  • simplify lists
  • remove unneeded escapes
  • remove attribute files
  • convert link:$$$.html to xref:$$$.adoc

@jamesnocentini
Copy link
Contributor

@graphitefriction those are wonderful changes!

One question, I noticed this PR adds explicit title ids. For example:

[#custom-auth]
== Custom Authentication

I'm wondering if it's really needed here?

@jamesnocentini
Copy link
Contributor

My preference would be to auto-generate the title IDs with hyphens instead of underscores. So I use the following on my local Antora config to do that:

asciidoc:
  attributes:
    idprefix: ''
    idseparator: '-'

@graphitefriction
Copy link
Contributor Author

@jamiltz I can definitely remove the explicit IDs (I added them out of habit because I tweak section titles constantly in my own writing and hate grepping through all the rest of the pages to fix the links I broke 😝 ) I agree 100% with putting the AsciiDoc attributes in the playbook. @mojavelinux is concerned that there may be a few ID collisions when applied across the whole migration set, so he's doing a test run. If there aren't massive conflicts, we'll add the attributes to the production and staging playbooks and remove the attributes from the document headers (DRY FTW!)

@jamesnocentini
Copy link
Contributor

Sounds good. Thanks! :)

@graphitefriction
Copy link
Contributor Author

@jamiltz I removed the explicit IDs and updated the PR. For now, the id attributes are in the headers of the targeted documents. Dan's got a big N1QL reference document with colliding IDs he has to debug before the attrs can be added to the playbook.

@jamesnocentini
Copy link
Contributor

Ok!

@jamesnocentini jamesnocentini merged commit 6d35db4 into couchbase:master Aug 21, 2018
@mojavelinux
Copy link
Contributor

Thanks James!

@mojavelinux mojavelinux deleted the clean-asciidoc branch August 21, 2018 19:17
ibsoln added a commit to ibsoln/docs-sync-gateway that referenced this pull request Oct 26, 2021
Set new issues list entries

(cherry picked from commit 91feb5b98468b1e3129cbce43237792e4efe75a2)
ibsoln added a commit to ibsoln/docs-sync-gateway that referenced this pull request Oct 26, 2021
Set new issues list entries

(cherry picked from commit 91feb5b98468b1e3129cbce43237792e4efe75a2)
ibsoln added a commit that referenced this pull request Oct 26, 2021
Set new issues list entries

(cherry picked from commit 91feb5b98468b1e3129cbce43237792e4efe75a2)
ibsoln added a commit to ibsoln/docs-sync-gateway that referenced this pull request Oct 26, 2021
https://issues.couchbase.com/browse/DOC-9307

DOC-9307 -- Release Notes for 2.8.3

https://issues.couchbase.com/browse/DOC-9307

DOC-9307 -- Release Notes for 2.8.3 (couchbase#2)
Feedback applied.

https://issues.couchbase.com/browse/DOC-9307

DOC-9307 -- Release Notes for 2.8.3 (couchbase#3)
Set new issues list entries

(cherry picked from commit df5120b27862a4b7880c3b69f0454f16cc80d22c)
(cherry picked from commit a812f46)

DOC-9307 -- Release Notes for 2.8.3 (couchbase#4)
Set new issues list entries
ibsoln added a commit to ibsoln/docs-sync-gateway that referenced this pull request Oct 26, 2021
https://issues.couchbase.com/browse/DOC-9307

DOC-9307 -- Release Notes for 2.8.3

https://issues.couchbase.com/browse/DOC-9307

DOC-9307 -- Release Notes for 2.8.3 (couchbase#2)
Feedback applied.

https://issues.couchbase.com/browse/DOC-9307

DOC-9307 -- Release Notes for 2.8.3 (couchbase#3)
Set new issues list entries

(cherry picked from commit df5120b27862a4b7880c3b69f0454f16cc80d22c)
(cherry picked from commit a812f46)

DOC-9307 -- Release Notes for 2.8.3 (couchbase#4)
Set new issues list entries

(cherry picked from commit 4d57eff)
ibsoln added a commit that referenced this pull request Oct 26, 2021
https://issues.couchbase.com/browse/DOC-9307

DOC-9307 -- Release Notes for 2.8.3

https://issues.couchbase.com/browse/DOC-9307

DOC-9307 -- Release Notes for 2.8.3 (#2)
Feedback applied.

https://issues.couchbase.com/browse/DOC-9307

DOC-9307 -- Release Notes for 2.8.3 (#3)
Set new issues list entries

(cherry picked from commit df5120b27862a4b7880c3b69f0454f16cc80d22c)
(cherry picked from commit a812f46)

DOC-9307 -- Release Notes for 2.8.3 (#4)
Set new issues list entries

(cherry picked from commit 4d57eff)
ibsoln added a commit that referenced this pull request Oct 26, 2021
https://issues.couchbase.com/browse/DOC-9307

DOC-9307 -- Release Notes for 2.8.3

https://issues.couchbase.com/browse/DOC-9307

DOC-9307 -- Release Notes for 2.8.3 (#2)
Feedback applied.

https://issues.couchbase.com/browse/DOC-9307

DOC-9307 -- Release Notes for 2.8.3 (#3)
Set new issues list entries

(cherry picked from commit df5120b27862a4b7880c3b69f0454f16cc80d22c)
(cherry picked from commit a812f46)

DOC-9307 -- Release Notes for 2.8.3 (#4)
Set new issues list entries
ibsoln added a commit to ibsoln/docs-sync-gateway that referenced this pull request Oct 26, 2021
Includes release notes an dminor supporting changes
https://issues.couchbase.com/browse/DOC-9307

DOC-9307 -- Release Notes for 2.8.3 (couchbase#597)

https://issues.couchbase.com/browse/DOC-9307

DOC-9307 -- Release Notes for 2.8.3

https://issues.couchbase.com/browse/DOC-9307

DOC-9307 -- Release Notes for 2.8.3 (couchbase#2)
Feedback applied.

https://issues.couchbase.com/browse/DOC-9307

DOC-9307 -- Release Notes for 2.8.3 (couchbase#3)
Set new issues list entries

(cherry picked from commit df5120b27862a4b7880c3b69f0454f16cc80d22c)
(cherry picked from commit a812f46)

DOC-9307 -- Release Notes for 2.8.3 (couchbase#4)
Set new issues list entries

(cherry picked from commit 122d375)

DOC-9307 -- Release Notes for 2.8.3
https://issues.couchbase.com/browse/DOC-9307

(cherry picked from commit 1f93a1bb4ad311081b53bc403b0562f6b26674be)
ibsoln added a commit to ibsoln/docs-sync-gateway that referenced this pull request Oct 26, 2021
Includes release notes an dminor supporting changes
https://issues.couchbase.com/browse/DOC-9307

DOC-9307 -- Release Notes for 2.8.3 (couchbase#597)

https://issues.couchbase.com/browse/DOC-9307

DOC-9307 -- Release Notes for 2.8.3

https://issues.couchbase.com/browse/DOC-9307

DOC-9307 -- Release Notes for 2.8.3 (couchbase#2)
Feedback applied.

https://issues.couchbase.com/browse/DOC-9307

DOC-9307 -- Release Notes for 2.8.3 (couchbase#3)
Set new issues list entries

(cherry picked from commit df5120b27862a4b7880c3b69f0454f16cc80d22c)
(cherry picked from commit a812f46)

DOC-9307 -- Release Notes for 2.8.3 (couchbase#4)
Set new issues list entries

(cherry picked from commit 122d375)

DOC-9307 -- Release Notes for 2.8.3
https://issues.couchbase.com/browse/DOC-9307

(cherry picked from commit 1f93a1bb4ad311081b53bc403b0562f6b26674be)
ibsoln added a commit to ibsoln/docs-sync-gateway that referenced this pull request Oct 27, 2021
Includes release notes an dminor supporting changes
https://issues.couchbase.com/browse/DOC-9307

DOC-9307 -- Release Notes for 2.8.3 (couchbase#597)

https://issues.couchbase.com/browse/DOC-9307

DOC-9307 -- Release Notes for 2.8.3

https://issues.couchbase.com/browse/DOC-9307

DOC-9307 -- Release Notes for 2.8.3 (couchbase#2)
Feedback applied.

https://issues.couchbase.com/browse/DOC-9307

DOC-9307 -- Release Notes for 2.8.3 (couchbase#3)
Set new issues list entries

(cherry picked from commit df5120b27862a4b7880c3b69f0454f16cc80d22c)
(cherry picked from commit a812f46)

DOC-9307 -- Release Notes for 2.8.3 (couchbase#4)
Set new issues list entries

(cherry picked from commit 122d375)

DOC-9307 -- Release Notes for 2.8.3
https://issues.couchbase.com/browse/DOC-9307

(cherry picked from commit 1f93a1bb4ad311081b53bc403b0562f6b26674be)
ibsoln added a commit that referenced this pull request Oct 28, 2021
* DOC-9307 -- Release Notes for 2.8.3
Includes release notes an dminor supporting changes
https://issues.couchbase.com/browse/DOC-9307

DOC-9307 -- Release Notes for 2.8.3 (#597)

https://issues.couchbase.com/browse/DOC-9307

DOC-9307 -- Release Notes for 2.8.3

https://issues.couchbase.com/browse/DOC-9307

DOC-9307 -- Release Notes for 2.8.3 (#2)
Feedback applied.

https://issues.couchbase.com/browse/DOC-9307

DOC-9307 -- Release Notes for 2.8.3 (#3)
Set new issues list entries

(cherry picked from commit df5120b27862a4b7880c3b69f0454f16cc80d22c)
(cherry picked from commit a812f46)

DOC-9307 -- Release Notes for 2.8.3 (#4)
Set new issues list entries

(cherry picked from commit 122d375)

DOC-9307 -- Release Notes for 2.8.3
https://issues.couchbase.com/browse/DOC-9307

(cherry picked from commit 1f93a1bb4ad311081b53bc403b0562f6b26674be)

* QF211027-1 -- Fix mislabelled image links

(cherry picked from commit 5b442f61eca3195a72646605ac90f97fa9b0dcec)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants